Skip to content

Conversation

@matiasdaloia
Copy link
Contributor

@matiasdaloia matiasdaloia commented Oct 9, 2025

Summary by CodeRabbit

  • New Features
    • Added data structures to support enhanced component/version/search and scan results for future scanning capabilities.
  • Chores
    • Applied GPL-2.0-or-later license headers across the codebase.
  • Documentation
    • Updated README and CI workflow to standardize and rename Make targets (test, lint, build targets, run).
  • Notes
    • No runtime behavior or public interfaces changed.

@matiasdaloia matiasdaloia self-assigned this Oct 9, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 9, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Adds GPL-2.0-or-later license headers across many source files, introduces new domain types (ComponentGroup, Version, SearchResult) in internal/domain/entities/component.go, and updates CI/Make target references in README and the release workflow. No runtime/control-flow changes detected.

Changes

Cohort / File(s) Summary of Changes
License headers added
cmd/import/main.go, internal/domain/entities/language.go, internal/domain/entities/metrics.go, internal/domain/entities/scan_request.go, internal/domain/entities/scan_response.go, internal/handler/scan_handler.go, internal/mapper/scan_mapper.go, internal/mapper/scan_mapper_impl.go, internal/repository/scan_repository.go, internal/repository/scan_repository_qdrant_impl.go, internal/service/scan_service.go, internal/service/scan_service_impl.go, internal/validation/validator.go
Inserted GPL-2.0-or-later / GPL v2+ SPDX header and copyright blocks. No functional or API changes.
New domain entities
internal/domain/entities/component.go
Added exported types: ComponentGroup (PURL, Name, Vendor, Versions, Rank, Order), Version (Version, Score), and SearchResult (Score, ID, Vendor, Component, Purl, Version, Rank) with JSON tags.
Docs & CI target updates
README.md, .github/workflows/release.yml
Renamed Make targets in README (e.g., unit_testtest, lint_locallint, build_amdbuild_amd64, added build_arm64, etc.). In release workflow, changed unit test step from make unit_test to make test.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I nibble on headers with hops and cheer,
New types appear where carrots used to peer.
Docs tidy their paths and CI takes a hop,
I thump for small changes — a tidy little crop. 🥕

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/mdaloia/add-license-headers

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 127461c and 6034000.

📒 Files selected for processing (16)
  • .github/workflows/release.yml (1 hunks)
  • README.md (3 hunks)
  • cmd/import/main.go (1 hunks)
  • internal/domain/entities/component.go (1 hunks)
  • internal/domain/entities/language.go (1 hunks)
  • internal/domain/entities/metrics.go (1 hunks)
  • internal/domain/entities/scan_request.go (1 hunks)
  • internal/domain/entities/scan_response.go (1 hunks)
  • internal/handler/scan_handler.go (1 hunks)
  • internal/mapper/scan_mapper.go (1 hunks)
  • internal/mapper/scan_mapper_impl.go (1 hunks)
  • internal/repository/scan_repository.go (1 hunks)
  • internal/repository/scan_repository_qdrant_impl.go (1 hunks)
  • internal/service/scan_service.go (1 hunks)
  • internal/service/scan_service_impl.go (1 hunks)
  • internal/validation/validator.go (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@matiasdaloia matiasdaloia merged commit e512240 into main Oct 9, 2025
2 of 3 checks passed
@matiasdaloia matiasdaloia deleted the chore/mdaloia/add-license-headers branch October 9, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants